Skip to content

Conversation

@mshavliuk
Copy link
Contributor

Fixes #4776

@mshavliuk mshavliuk requested a review from a team as a code owner September 8, 2025 10:48
@mshavliuk mshavliuk force-pushed the fix/optional-langchain branch from 4afec9c to 33629d3 Compare September 8, 2025 10:49
@mshavliuk mshavliuk force-pushed the fix/optional-langchain branch from 33629d3 to 0fbebf2 Compare September 8, 2025 15:33
@codecov
Copy link

codecov bot commented Sep 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.63%. Comparing base (567a91a) to head (0fbebf2).
⚠️ Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4778      +/-   ##
==========================================
+ Coverage   84.61%   84.63%   +0.01%     
==========================================
  Files         158      158              
  Lines       16376    16378       +2     
  Branches     2809     2808       -1     
==========================================
+ Hits        13857    13861       +4     
+ Misses       1693     1692       -1     
+ Partials      826      825       -1     
Files with missing lines Coverage Δ
sentry_sdk/integrations/langchain.py 64.40% <100.00%> (+0.41%) ⬆️

... and 1 file with indirect coverage changes

@mshavliuk
Copy link
Contributor Author

Fixed in #4783

@mshavliuk mshavliuk closed this Sep 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

LangchainIntegration breaks if only langchain-core is installed

1 participant